Skip to content

fix: tolerate missing ResourceClaim API during PCLQ reconcile and delete#610

Merged
sanjaychatterjee merged 1 commit into
ai-dynamo:mainfrom
SAY-5:fix-resourceclaim-no-match
May 18, 2026
Merged

fix: tolerate missing ResourceClaim API during PCLQ reconcile and delete#610
sanjaychatterjee merged 1 commit into
ai-dynamo:mainfrom
SAY-5:fix-resourceclaim-no-match

Conversation

@SAY-5
Copy link
Copy Markdown
Contributor

@SAY-5 SAY-5 commented May 12, 2026

What type of PR is this?

/kind bug

What this PR does / why we need it:

PodClique reconcile and delete now treat meta.IsNoMatchError from ResourceClaim list/delete as "nothing to do" instead of a hard failure, so cleanup is not blocked on clusters that do not serve the resource.k8s.io ResourceClaim API.

Which issue(s) this PR fixes:

Fixes #609

Special notes for your reviewer:

If the API is absent there can be no ResourceClaim objects to reconcile or delete, so skipping is safe. Added regression tests using an interceptor that returns NoKindMatchError.

Does this PR introduce a API change?

NONE

Additional documentation e.g., enhancement proposals, usage docs, etc.:


@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 12, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@SAY-5 SAY-5 force-pushed the fix-resourceclaim-no-match branch from 185e4ad to 545d5ab Compare May 13, 2026 18:52
@SAY-5
Copy link
Copy Markdown
Contributor Author

SAY-5 commented May 13, 2026

Added a V(1) debug log on the swallowed NoMatchError in Delete with namespace, podClique, and err. Single-commit force push, +2/-1.

@danbar2 danbar2 force-pushed the fix-resourceclaim-no-match branch from 545d5ab to dcc65d6 Compare May 17, 2026 08:03
@sanjaychatterjee sanjaychatterjee force-pushed the fix-resourceclaim-no-match branch from dcc65d6 to 5bbeac9 Compare May 18, 2026 07:19
Signed-off-by: Sai Asish Y <say.apm35@gmail.com>
@sanjaychatterjee sanjaychatterjee force-pushed the fix-resourceclaim-no-match branch from 5bbeac9 to ff25c92 Compare May 18, 2026 14:55
@sanjaychatterjee sanjaychatterjee merged commit e359cc0 into ai-dynamo:main May 18, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PodClique cleanup fails when resource.k8s.io/v1 ResourceClaim API is unavailable

6 participants